-
Notifications
You must be signed in to change notification settings - Fork 17
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove Unnecessary Non-Terminal Key checks #248
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #248 +/- ##
==========================================
+ Coverage 92.23% 92.78% +0.54%
==========================================
Files 42 45 +3
Lines 2564 3255 +691
Branches 298 300 +2
==========================================
+ Hits 2365 3020 +655
- Misses 195 230 +35
- Partials 4 5 +1 ☔ View full report in Codecov by Sentry. |
0f7fc7d
to
675199a
Compare
675199a
to
896ae0c
Compare
@@ -59,6 +59,7 @@ contract PermissionlessNodeRegistry is | |||
//mapping of operator address with nodeELReward vault address | |||
mapping(uint256 => address) public override nodeELRewardVaultByOperatorId; | |||
mapping(uint256 => address) public proposedRewardAddressByOperatorId; | |||
uint256 public maxKeyPerOperator; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maxKeysPerOperator
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
* @notice update the max validator per operator value | ||
* @dev only `MANAGER` role can update | ||
*/ | ||
function updateMaxKeyPerOperator(uint256 _maxKeyPerOperator) external { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please get this reviewed by @batphonghan as well.
No description provided.